fix: check masquerade on instructor dashboard tab acess check #601
Conversation
ArturGaspar
force-pushed
the
artur/hide-instructor-tab-in-track-masquerade
branch
from
7c816bb
to
7a1c242
Compare
|
Sandbox deploy request received. Deployment will start soon. |
|
Sandbox deployment started. |
ArturGaspar
force-pushed
the
artur/hide-instructor-tab-in-track-masquerade
branch
from
7a1c242
to
251c959
Compare
Agrendalath
changed the base branch from
opencraft-release/palm.1-echo
to
opencraft-release/palm.1
Agrendalath
force-pushed
the
artur/hide-instructor-tab-in-track-masquerade
branch
from
251c959
to
88e54f9
Compare
Agrendalath
force-pushed
the
artur/hide-instructor-tab-in-track-masquerade
branch
from
88e54f9
to
5ce151c
Compare
There was a problem hiding this comment.
👍
- I tested this: checked that the diff matches the upstream PR
- I read through the code
- I checked for accessibility issues: n/a
- Includes documentation: n/a
- I made sure any change in configuration variables is reflected in the corresponding client's
configuration-securerepository: n/a
|
Sandbox deployment successful. Sandbox LMS is available at pr-601-0e7f8a.staging.do.opencraft.hosting |
|
Sandbox deployment started. |
|
Sandbox deployment successful. Sandbox LMS is available at pr-601-0e7f8a.staging.do.opencraft.hosting |
|
Sandbox deployment started. |
|
Sandbox deployment successful. Sandbox LMS is available at pr-601-0e7f8a.staging.do.opencraft.hosting |
|
Sandbox update request received. Deployment will start soon. |
Backports openedx#33684.
Description
Check also masquerading role to determine if instructor tab is enabled, in order to hide it when a user with acess is masquerading as a student role without a specific different user.
Impacts Staff and Instructor users.
Testing instructions
Deadline
None
Other information
Private-Ref: https://tasks.opencraft.com/browse/BB-8087